overview: Only keep ourselves in the overview on button events

This prevents using Control to quickly launch a new instance of
an app in search results or with keyboard navigation.

https://bugzilla.gnome.org/show_bug.cgi?id=695801

diff --git a/js/ui/overview.js b/js/ui/overview.js
index 4fdcd02a2debd2b94717a97722c00962dac7714c..33cc478abe55ab4c46077a85b16ddee951fc888d 100644 (file)
--- a/js/ui/overview.js
+++ b/js/ui/overview.js
@@ -572,8 +572,14 @@ const Overview = new Lang.Class({
             return;
 
         let event = Clutter.get_current_event();
-        if (event && (event.get_state() & Clutter.ModifierType.CONTROL_MASK) != 0)
-            return;
+        if (event) {
+            let type = event.type();
+            let button = (type == Clutter.EventType.BUTTON_PRESS ||
+                          type == Clutter.EventType.BUTTON_RELEASE);
+            let ctrl = (event.get_state() & Clutter.ModifierType.CONTROL_MASK) != 0;
+            if (button && ctrl)
+                return;
+        }
 
         this._animateNotVisible();